Skip to content

Conversation

@sftwrdvlpr
Copy link
Contributor

@sftwrdvlpr sftwrdvlpr commented Nov 8, 2025

Description

  • Updated airbase to 325
  • Updated airlift BOM to 372
  • Updated okhttp3 to 5.3.0
  • Added okhttp3 BOM
  • Added okhttp-jvm dependency
  • Updated jersey-server to 3.1.11
  • Updated jetty to 12.1.3

Additional context and related issues

Fixes issue #571 by setting configured max-request-header-size and max-response-header-size parameters on the HttpClient.

Release notes

( ) This is not user-visible or is docs only, and no release notes are required.
(x) Release notes are required, with the following suggested text:

* Set `max-request-header-size` and `max-response-header-size` parameters on the HttpClient to support large HTTP headers.

@cla-bot cla-bot bot added the cla-signed label Nov 8, 2025
@mosabua
Copy link
Member

mosabua commented Nov 8, 2025

Thank you so much for submitting this PR. I think we will try to get this in before the next release (see #792 ) and also include the upgrade to Java 25.

I am a bit surprised that not more changes are needed .. last time I looked at that upgrade (only for a few minutes and a while ago but anyway) I kinda remember to running into other things, but maybe the bigger jump fixed things.

@sftwrdvlpr
Copy link
Contributor Author

Yes, usually such a big gap between versions doesn't go that smoothly, though I was prepared for bigger changes.

@sftwrdvlpr
Copy link
Contributor Author

@mosabua just out of curiosity, what is the planned release date?

Copy link
Member

@mosabua mosabua left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good to go. I will merge and send follow up with Java 25 and further upgrades.

@mosabua mosabua merged commit 3267d06 into trinodb:main Nov 29, 2025
2 checks passed
@github-actions github-actions bot added this to the 17 milestone Nov 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

3 participants